Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Prod] Modify transaction wrapper; font change; remove ElasticSearch; historical goal status change #2266

Merged
merged 42 commits into from
Jul 12, 2024
Merged

[Prod] Modify transaction wrapper; font change; remove ElasticSearch; historical goal status change #2266

merged 42 commits into from
Jul 12, 2024

Conversation

kryswisnaskas
Copy link
Collaborator

@kryswisnaskas kryswisnaskas commented Jul 11, 2024 •
edited
Loading

Description of change

  • 3144: Adjust transaction wrapper and catch logic.
  • 3112: Remove ElasticSearch.
  • 2795: Historical goal status change import migration, sourced from audit log.
  • 2907: Update font for something went wrong and hide avatar on error.

How to test

Issue(s)

Checklists

Every PR

  • Meets issue criteria
  • JIRA ticket status updated
  • Code is meaningfully tested
  • Meets accessibility standards (WCAG 2.1 Levels A, AA)
  • [n/a] API Documentation updated
  • [n/a] Boundary diagram updated
  • [n/a] Logical Data Model updated
  • [n/a] Architectural Decision Records written for major infrastructure decisions
  • [n/a] UI review complete

Before merge to main

  • [n/a] OHS demo complete
  • Ready to create production PR

Production Deploy

  • Staging smoke test completed

After merge/deploy

  • Update JIRA ticket status

nvms and others added 30 commits June 11, 2024 11:55
@nvms
initial pass
88f0c0c
@nvms
Merge branch 'main' into jp/2795/GSC-migration
7ab4c77
@nvms
update ldm, update model, change mig date
1402353
@nvms
replace tabs with spaces
0be561a
@nvms
Garrett's change to prevent re-insertion of existing records.
0803f09
@nvms
Merge branch 'main' into jp/2795/GSC-migration
669649a
@nvms
mv migration
5015cc6
@nvms
userRoles is nullable now
b5f0304
@nvms
mv migration
7e4705b
@nvms
make reason nullable
1601fe5
@nvms
nullable columns, nullif user id, left join on Goals
ed2c42d
@nvms
fix datatype, update ldm
288bba8
@nvms
rm src/lib/awsElasticSearch
87abe33
@nvms
rm constants and hooks
65d87ea
@nvms
rm export
2d0bda2
@nvms
bump @ttahub/common
7ba6007
@nvms
rm src/routes/search
9120e87
@nvms
rm worker job
1e232dc
@nvms
finish removing src/routes/search
9d3e507
@nvms
remove packages
186f47f
@nvms
remove terraform stuff
91aec02
@nvms
remove openapi stuff
db4ffa8
@nvms
remove containers
188dee0
@nvms
remove CI executor
6a57ff9
@nvms
remove from cf manifest
c3c1c75
@nvms
update readme
71c1979
@nvms
update yarn-audit-known-issues
863d5e4
@nvms
deploy to sandbox
b14d499
@nvms
rm more refs
d643209
@AdamAdHocTeam
font updates per Kelly
bc2a51d
nvms and others added 12 commits July 10, 2024 16:20
@kryswisnaskas kryswisnaskas changed the title [Prod] Modify transaction wrapper; font change [Prod] Modify transaction wrapper; font change; remove ElasticSearch; historical goal status change Jul 12, 2024
@kryswisnaskas kryswisnaskas marked this pull request as ready for review July 12, 2024 13:36
@Jones-QuarteyDana Jones-QuarteyDana merged commit 689b01f into production Jul 12, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jones-QuarteyDana Jones-QuarteyDana Jones-QuarteyDana approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@kryswisnaskas @Jones-QuarteyDana @nvms @AdamAdHocTeam